docs: refresh architecture/ deep-dives for the 0.10.0 surfaces#98
Merged
Conversation
Promote the pass-3 / 0.10.0 conclusions into the architecture/*.md deep-dives so they match the shipped code (the audit updated CLAUDE.md inline; these lagged): - dlq.md: last_exception_renderer redaction hook (F3-01); validate_schema's extra pg_catalog probes (partial-index predicate, timer_id_uq indisunique, lease CHECK). - drain.md: timed-out drain emits WARNING + drain_timeout metric (F1-04); broker.stop sets running=False before the gather (F1-03); new drain_timeout regression test. - timers.md: activate_at on single publish is a client literal (not make_interval), worker-clock NOTIFY (F2-04/F6-13); timer_id dedup is one-live-row, not once-ever (F2-06). - test-broker.md: fake session leniency diverges from production (F4-09), OutboxResponse validation still fires; shared _claim_fake_row claim mechanics (F1-08). relay.md (F5-01) and metrics.md (drain_timeout) were already current. Docs-only; just lint clean. Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Documentation-only. Promotes the pass-3 / 0.10.0 conclusions into the
architecture/*.mddeep-dives so they match the shipped code — the audit updatedCLAUDE.md(andrelay.md/metrics.md) inline, but the other deep-dives lagged.last_exception_rendererredaction hook (F3-01);validate_schema's extrapg_catalogprobes beyond the alembic diff (partial-index predicate,timer_id_uqindisunique,<table>_lease_ckCHECK).drain_timeoutmetric (F1-04);OutboxBroker.stopsetsrunning=Falsebefore the gather (F1-03); added the new regression test to the list.activate_aton singlepublishis a client literal (notmake_interval), so its NOTIFY future-dating is worker-clock-relative (F2-04/F6-13);timer_iddedup is "one live row", not once-ever (F2-06).sessionleniency diverges from production'sTypeError(F4-09) andOutboxResponsevalidation still fires; the shared_claim_fake_rowclaim mechanics betweenfetchand_sync_dispatch(F1-08).relay.md(F5-01) andmetrics.md(drain_timeout) were already refreshed during the audit.Verification
just lintclean (eof + ruff + ty). Docs-only — no code/tests touched.This is the optional architecture sweep flagged after the release; with it merged, the deep-dives,
CLAUDE.md, and the shipped 0.10.0 code are all in agreement.🤖 Generated with Claude Code